Skip to content

Light Freshness Edit: ASP.NET - Startup, gRPC, YARP, SPA#37053

Open
GitHubber17 wants to merge 2 commits intodotnet:mainfrom
GitHubber17:568764-f
Open

Light Freshness Edit: ASP.NET - Startup, gRPC, YARP, SPA#37053
GitHubber17 wants to merge 2 commits intodotnet:mainfrom
GitHubber17:568764-f

Conversation

@GitHubber17
Copy link
Copy Markdown
Contributor

@GitHubber17 GitHubber17 commented Apr 24, 2026

@GitHubber17 GitHubber17 changed the title refresh content + include Light Freshness Edit: ASP.NET - Startup, gRPC, YARP, SPA Apr 24, 2026
@GitHubber17 GitHubber17 marked this pull request as ready for review April 24, 2026 05:27
@GitHubber17
Copy link
Copy Markdown
Contributor Author

GitHubber17 commented Apr 24, 2026

Hi @wadepickett - Refreshed topics ready for your review. See top comment for details. Thanks

Cc @cmastr

Comment thread aspnetcore/grpc/basics.md

* [Overview for gRPC on .NET](xref:grpc/index)
* [Create a .NET gRPC client and server in ASP.NET Core](xref:tutorials/grpc/grpc-start)
* [Call gRPC services with the .NET client](xref:grpc/client)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* [Call gRPC services with the .NET client](xref:grpc/client)
* [gRPC services with ASP.NET Core](xref:grpc/aspnetcore)
* [Call gRPC services with the .NET client](xref:grpc/client)

Added gRPC Services back in.

Comment thread aspnetcore/grpc/basics.md
For more information on the syntax of protobuf files, see [Create Protobuf messages for .NET apps](xref:grpc/protobuf).

For example, consider the *greet.proto* file used in [Get started with gRPC service](xref:tutorials/grpc/grpc-start):
Consider the _greet.proto_ file used in the tutorial, [Create a gRPC client and server in ASP.NET Core](xref:tutorials/grpc/grpc-start)s:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Consider the _greet.proto_ file used in the tutorial, [Create a gRPC client and server in ASP.NET Core](xref:tutorials/grpc/grpc-start)s:
Consider the _greet.proto_ file used in the tutorial, [Create a gRPC client and server in ASP.NET Core](xref:tutorials/grpc/grpc-start):

Removed typo "s".

@@ -1,12 +1,14 @@
---
title: App startup in ASP.NET Core
title: App Startup in ASP.NET Core
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
title: App Startup in ASP.NET Core
title: App startup in ASP.NET Core

Per repo guidelines, titles use sentence case. First word and proper nouns capitalized. In this case startup is not a proper noun.

uid: fundamentals/servers/yarp/getting-started
title: YARP Getting Started with YARP
description: YARP Getting Started with YARP
title: Get Started with YARP
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
title: Get Started with YARP
title: Get started with YARP

Good catch, multiple YARPS! That awful we had that in there. Minor fix: Sentence case for titles.

* To create a pipeline of `Configure` methods. [IStartupFilter.Configure](xref:Microsoft.AspNetCore.Hosting.IStartupFilter.Configure%2A) can set a middleware to run before or after middleware added by libraries.

An `IStartupFilter` implementation implements <xref:Microsoft.AspNetCore.Hosting.StartupBase.Configure%2A>, which receives and returns an `Action<IApplicationBuilder>`. An <xref:Microsoft.AspNetCore.Builder.IApplicationBuilder> defines a class to configure an app's request pipeline. For more information, see [Create a middleware pipeline with IApplicationBuilder](xref:fundamentals/middleware/index#create-a-middleware-pipeline-with-iapplicationbuilder).
An `IStartupFilter` instance implements the <xref:Microsoft.AspNetCore.Hosting.StartupBase.Configure%2A>method, which receives and returns an `Action<IApplicationBuilder>`. An <xref:Microsoft.AspNetCore.Builder.IApplicationBuilder> defines a class to configure an app's request pipeline. For more information, see [Create a middleware pipeline with IApplicationBuilder](xref:fundamentals/middleware/index#create-a-middleware-pipeline-with-iapplicationbuilder).
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett Apr 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
An `IStartupFilter` instance implements the <xref:Microsoft.AspNetCore.Hosting.StartupBase.Configure%2A>method, which receives and returns an `Action<IApplicationBuilder>`. An <xref:Microsoft.AspNetCore.Builder.IApplicationBuilder> defines a class to configure an app's request pipeline. For more information, see [Create a middleware pipeline with IApplicationBuilder](xref:fundamentals/middleware/index#create-a-middleware-pipeline-with-iapplicationbuilder).
An `IStartupFilter` implementation provides a <xref:Microsoft.AspNetCore.Hosting.StartupBase.Configure%2A> method that receives and returns an Action<IApplicationBuilder>. An <xref:Microsoft.AspNetCore.Builder.IApplicationBuilder> defines a class to configure an app's request pipeline. For more information, see [Create a middleware pipeline with IApplicationBuilder](xref:fundamentals/middleware/index#create-a-middleware-pipeline-with-iapplicationbuilder).

"imlemenation implements" sounds goofy. Alhtough technicaly correct if I squint, it needed an improvment certtainly. Changing to "instance implements" is technically incorrect, since instances don't implement interfaces, classes do. See what you think of my suggestion above.

Also a minor item fixed: missing space added after a bracket.

The following app startup code supports several app types:

* [Blazor Web Apps](xref:blazor/index)
* [Blazor web apps](xref:blazor/index#build-a-full-stack-web-app-with-blazor)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* [Blazor web apps](xref:blazor/index#build-a-full-stack-web-app-with-blazor)
* [Blazor Web Apps](xref:blazor/index#build-a-full-stack-web-app-with-blazor)

Proper noun here I think.

Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work, thanks! I made a few corrections/suggestions inline to the articles.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants